Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for BINTEXT and BOOLEAN #254

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

he-is-harry
Copy link
Contributor

  • Added support for BINTEXT and BOOLEAN for data format version 7
    • BINTEXT is sent and received as buffers
    • BOOLEAN can be sent as strings, numbers, and boolean and received as boolean
  • Modified data type integration tests to skip tests when the data format version is lower than required
  • Added integration tests for BINTEXT and BOOLEAN
  • Added unit tests for BOOLEAN

- Added support for BINTEXT and BOOLEAN for data format version 7
    - BINTEXT is sent and received as buffers
    - BOOLEAN can be sent as strings, numbers, and boolean and received as boolean
- Modified data type integration tests to skip tests when the data format version is lower than required
- Added integration tests for BINTEXT and BOOLEAN
- Added unit tests for BOOLEAN
- Added spatialTypes option to allow strings to be sent for spatial types
    - With spatialTypes=0 (default), strings inputted for ST_GEOMETRY and ST_POINT are interpreted as hex
    - With spatialTypes=1, strings inputted for ST_GEOMETRY and ST_POINT can be encoded in utf8 or cesu8
    - Buffers can be sent in any of these options and will be interpreted as binary
- Added integration tests to check inserting and fetching large ST_GEOMETRY data
- Added some integration tests to check the spatialTypes=1 behaviour
- Changed the Writer to return an error message of 'Argument must be a string or Buffer' when an invalid type is entered for spatial types
- Added some hex string and buffer inputs to ST_POINT tests
- Changed some inputs for integration tests to describe what is tested more accurately
- Merge changes for data format version 5 into this branch to resolve merge conflicts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant